Skip to content

net: openthread: rpc: remove unneeded response check#26095

Merged
nordicjm merged 1 commit into
nrfconnect:mainfrom
Damian-Nordic:ot_rpc_unneeded_check
Dec 16, 2025
Merged

net: openthread: rpc: remove unneeded response check#26095
nordicjm merged 1 commit into
nrfconnect:mainfrom
Damian-Nordic:ot_rpc_unneeded_check

Conversation

@Damian-Nordic
Copy link
Copy Markdown
Contributor

... for an RPC call that has an empty response.

... for an RPC call that has an empty response.

Signed-off-by: Damian Krolik <damian.krolik@nordicsemi.no>
@Damian-Nordic Damian-Nordic requested a review from a team as a code owner December 4, 2025 14:19
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 4, 2025
@NordicBuilder NordicBuilder requested review from a team December 4, 2025 14:19
@Damian-Nordic Damian-Nordic removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Dec 4, 2025
@NordicBuilder
Copy link
Copy Markdown
Contributor

NordicBuilder commented Dec 4, 2025

CI Information

To view the history of this post, click the 'edited' button above
Build number: 1

Inputs:

Sources:

sdk-nrf: PR head: 0bdc6c6e6d08a4782a98b077749475cf81318433

more details

sdk-nrf:

PR head: 0bdc6c6e6d08a4782a98b077749475cf81318433
merge base: af76f4a9447fb15eb8f06b38bcf60a97a6da3ea9
target head (main): af76f4a9447fb15eb8f06b38bcf60a97a6da3ea9
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (1)
subsys
│  ├── net
│  │  ├── openthread
│  │  │  ├── rpc
│  │  │  │  ├── server
│  │  │  │  │  │ ot_rpc_udp.c

Outputs:

Toolchain

Version: 43683a87ea
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:43683a87ea_5ea73affbf

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 786
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-thread-main
    • ✅ test-fw-nrfconnect-ps-main
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@nordicjm nordicjm merged commit 591b6a7 into nrfconnect:main Dec 16, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants